Skip to content

Add linewidth support to legend inputs#658

Open
K-Mirembe-Mercy wants to merge 2 commits intoUltraplot:mainfrom
K-Mirembe-Mercy:clean-linewidth-fix
Open

Add linewidth support to legend inputs#658
K-Mirembe-Mercy wants to merge 2 commits intoUltraplot:mainfrom
K-Mirembe-Mercy:clean-linewidth-fix

Conversation

@K-Mirembe-Mercy
Copy link

Summary

Adds support for "linewidth" in legend inputs.

Changes

  • Passes "linewidth" from kwargs into "_LegendInputs"
  • Allows users to control legend frame linewidth

Notes

  • Minimal change, no breaking behavior

@cvanelteren
Copy link
Collaborator

This PR is not complete yet. Have a look at our previous commits on how they are structured and our CONTRIBUTING docs or https://ultraplot.readthedocs.io/en/latest/contributing.html

A PR will have the following elements

  • Does it fix an issue?
  • Does it contain test for the patched code?
  • Does it need documentation?
  • Does it need examples in the gallery?

The PR here does not follow the same pattern for the other line widths that were introduced for semantic legends.

@cvanelteren
Copy link
Collaborator

Also do we need to close #654 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants